Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feat/CK-209] 새로고침 시 로그인 페이지로 이동하는 버그를 해결한다. #169

Merged
merged 2 commits into from
Oct 1, 2023

Conversation

sh981013s
Copy link
Collaborator

📌 작업 이슈 번호

CK-209

✨ 작업 내용

  • 기존 로그인 시 /me 요청을 통해 나라는 유저의 정보를 가져오쥬? 그리고 받아온 정보를 전역 컨텍스트에 저장 후 지속적으로 사용합니다.
    허나 문제는 새로고침 시 전역 컨텍스트에 저장되어 있는 나라는 유저의 정보가 초기화가 되기에, 미리 가지고 있던 accessToken 을 사용하여 다시 유저 정보를 가져온다 해도 가져오기 전 빈 시간동안은 유저 정보가 없게 됩니다. 그 시점에, 유저가 private router(유저가 로그인 되어있는지 체크) 로 감싸진 위치에 있다면 강제적으로 로그인 페이지로 이동하게 되었던 상황임다.

  • 그래서 throttle 이나 debounce 를 걸어 시간을 벌어야 할지, 고민 중 성공은 했으나, 정확한 조건을 정의할 수 없었고 성공률이 100퍼센트는 아니라고 생각했기에, 단순 유저가 로그인 되어있는지 확인하는 로직에 accessToken 까지 체크하는 로직을 넣어 다시 유저 정보를 불러올 수 있는 시간을 벌 수 있게 했슴다.

💬 리뷰어에게 남길 멘트

잘 부탁드려요!

🚀 요구사항 분석

  • 새로고침 시 로그인 페이지로 강제 이동하는 버그 픽스

@sh981013s sh981013s added fix ⚒️ Something isn't working FE 👨‍👨‍👧 FrontEnd labels Sep 28, 2023
@sh981013s sh981013s self-assigned this Sep 28, 2023
Copy link
Collaborator

@NaveOWO NaveOWO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

파일 분리해줘서 고마워용 엉!!
해야지 했는데 너무 귀찮았는데..!! 엑세스 토큰 추가해준거 확인이요!! 똑똑하다👍👍

Copy link
Collaborator

@jw-r jw-r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

client가 가진 토큰만 존재해도 일단 허용을 하는군요!
그 후에 토큰 관련 요청과 에러 핸들링 수행되니 유효하지 않은 토큰이라면 그 때 비로소 로그인 페이지로 이동하겠네요!

너무 좋은 것 같습니다🤩
고생하셨어요!!

@sh981013s sh981013s merged commit ce9c5c7 into develop-client Oct 1, 2023
1 check passed
@sh981013s sh981013s deleted the feature/CK-209 branch October 1, 2023 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 👨‍👨‍👧 FrontEnd fix ⚒️ Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants